Conversation
| gem 'slim-rails' | ||
|
|
||
| # Serializers | ||
| gem 'jbuilder' |
There was a problem hiding this comment.
Не очень понравился выбор в виде jbuilder - он конечно работает из коробки и хорошо описан, но уж больно он медленный по сравнению с другими сериалайзерами. У нас он сейчас на проекте, очень страдаем от его скорости и везде где только можно переходим на alab (у которого Си под капотом)
| @@ -0,0 +1,9 @@ | |||
| class BooksController < ApplicationController | |||
| def index | |||
| page = params[:page].to_i.positive? ? params[:page].to_i : 1 | |||
There was a problem hiding this comment.
Методу page(), который добавляет гем Kaminary можно сразу передать params[:page] - он обработает все случаи: отрицательные значение, nil, просто вандальные параметры. Т.е. можем избавиться вот от этой строки
page = params[:page].to_i.positive? ? params[:page].to_i : 1| username: <%= ENV.fetch('POSTGRES_USER') { 'igorsimdyanov' } %> | ||
| password: <%= ENV.fetch('POSTGRES_PASSWORD') { '' } %> | ||
| username: <%= ENV.fetch('POSTGRES_USER') { 'postgres' } %> | ||
| password: <%= ENV.fetch('postgres') { '' } %> |
There was a problem hiding this comment.
В config/database.yml помоему опечатка в следующей строке
password: <%= ENV.fetch('postgres') { '' } %>Тут лучше оставить значение переменной окружения POSTGRES_PASSWORD, если нужно установить пароль, то лучше это сделать в блоке метода fetch
password: <%= ENV.fetch('POSTGRES_PASSWORD') { 'postgres' } %>2 participants
No description provided.